Skip to content

Added multi-pin TDM support.#470

Open
ccrome wants to merge 1 commit intoPaulStoffregen:masterfrom
ccrome:ccrome/tdm-multipin
Open

Added multi-pin TDM support.#470
ccrome wants to merge 1 commit intoPaulStoffregen:masterfrom
ccrome:ccrome/tdm-multipin

Conversation

@ccrome
Copy link
Copy Markdown
Contributor

@ccrome ccrome commented Jan 18, 2024

This allows you use use more than one TX or RX pin in TDM mode, allowing for up to 64 channels of record or playback data

This fix goes hand-in-hand with the multi-channel USB support. PaulStoffregen/cores#732

input_tdm: every odd channel had every other sample swapped In every odd channel in TDM input (1, 3, 5, 7, 9, 11, 13, 15), every other word was swapped due to an incorrect copy from 32-bits to
16-bits. This fix corrects the odd channels. The Shift-by zeros and the
extraneous logical ands are there for clarity, and I verified they
don't end up affecting final code optimization as long as optimization
is turned on.

This allows you use use more than one TX or RX pin in TDM mode,
allowing for up to 64 channels of record or playback data

This fix goes hand-in-hand with the multi-channel USB support.
PaulStoffregen/cores#732

input_tdm: every odd channel had every other sample swapped
In every odd channel in TDM input (1, 3, 5, 7, 9, 11, 13, 15), every
other word was swapped due to an incorrect copy from 32-bits to
16-bits. This fix corrects the odd channels.   The Shift-by zeros  and the
extraneous logical ands are there for clarity, and I verified they
don't end up affecting final code optimization as long as optimization
is turned on.
@ccrome ccrome force-pushed the ccrome/tdm-multipin branch from 8a494ed to 28d3ab4 Compare January 18, 2024 23:26
@ccrome ccrome marked this pull request as ready for review January 18, 2024 23:26
@h4yn0nnym0u5e
Copy link
Copy Markdown
Contributor

This is non-functional, as the changes to TDM output are not complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants